Repository navigation
feat(work-items): integrate at least daily in decompose's window bar - #8382
Conversation
…cadence The window bar now also asks that a slice can merge to the mainline at least daily, as practice guidance (DORA trunk-based development, Fowler's Continuous Integration), with no hours estimate and no day cap. Adds an eval case and a changelog fragment. Refs #8328 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The paid run showed every with-arm reply stating the floor as "merge it the same day" or "merge within a day", which the judge passed and which meets the skill's "at least daily" floor, while the regex demanded the literal phrase. The regex now also accepts "the same day" and "within a day", with labeled samples for both and a negated same-day fail. Refs #8328 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
claude-security-review reviewed No security issues found in the incremental change since 57f73d2.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 57f73d2360
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
claude-review reviewed No findings on the incremental change since 57f73d2. I read
|
The DORA and Fowler pointers now link the exact sections the row cites, as the skill-bodies rule requires of a docs-page pointer. The daily-integration regex spells out its integrate forms so the typos gate stops reading "integrat" as a misspelling, and both sample files end in a newline for editorconfig. Refs #8328 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Closes #8328
Summary
/work-items:decompose's window-bar row keeps the one-fresh-context-window bar and adds that a vertical slice can merge to the mainline at least daily. The cadence is practice guidance, the merge floor, not an estimate: the row states no hours unit, no time estimate and no day cap.Fix
plugins/work-items/skills/decompose/SKILL.md: the window-bar row adds the daily-integration clause, forbids a time estimate or day cap beside the existing token-budget ban, and points at DORA, trunk-based development and Fowler, Continuous Integration with an as-of date and recheck trigger.plugins/work-items/evals/decompose-slice-checks-daily-integration: an LLM judge on the window-bar verdict, a regex that the reply states daily integration, and a regex that it gives no hours estimate, with labeled samples for both regexes..changes/work-items/feat-8328-decompose-daily-integration-6b1e4d9a.md.Acceptance criteria
grep -c "at least daily"on the decompose SKILL.md returns 1;grep -ciE "[0-9]+ hours"returns 0.case.yamldir under the work-items plugin'sevals/): a slice sized to the window bar is also checked for daily integration, with no hours estimate.bash scripts/check-changed-skills.sh origin/mainexits 0 (PASS, 0 errors, 4 pre-existing size and style warnings).Verification
scripts/check-affected.sh --tests: 7 passed, 2 skipped (shellcheck, ruff: no files of that kind changed).validate-cases.py plugins/work-items/evals: exit 0 (two WARN lines on the unrelateddecompose-no-container-carries-briefcase).Paid eval,
claude plugin evalon Claude Code 2.1.303, model claude-opus-5-5, judge opus, 3 runs per arm, casedecompose-slice-checks-daily-integration, plugin copied outside the repository:The judge voted unanimously in all 6 runs of each pass. On main the with-arm applies the window bar but never states daily integration, so the judge and the daily regex fail; on the branch all three graders pass in every with-arm run. Noise report:
n too small to call(one case; 3 or more needed for an interval), so this measures the case, not a suite-wide gain. List-price cost about 0.63 USD for the branch pass.Left out: judge calibration for
applies-window-bar(it has no labeled samples, so it is reported uncalibrated); the validity gate'sskill firedwarning (the prompt loads the skill by its slash name, which the with-arm replies quote, and the case has notool_usedgrader); main was measured at the branch base, not the main checkout, which was 81 commits behind.Related
🤖 Generated with Claude Code